[ADD] accounting/l10n_co: adding automatic DIAN enablement process#18026
[ADD] accounting/l10n_co: adding automatic DIAN enablement process#18026masi-odoo wants to merge 1 commit into
Conversation
masi-odoo
commented
May 21, 2026
masi-odoo
commented
- Added section on automatic DIAN enablement process for electronic invoicing, with new video reference
- Eliminated previous video on the topic which is not up to date
|
|
afma-odoo
left a comment
afma-odoo
left a comment
There was a problem hiding this comment.
Thank you @masi-odoo for your work on this Colombian localization page
I added some comments/suggestions. If you need more information, just let me know.
Thanks!
|
Hey @afma-odoo thanks for reviewing this! I applied the suggestions they were really good. Please take another look and let me know if there is anything else to be done before continuing! |
afma-odoo
left a comment
afma-odoo
left a comment
There was a problem hiding this comment.
Hi @masi-odoo Thank you for the updates
I misspelled :guilabel: in my review (sorry about that!), so I took the opportunity to add a few suggestions.
I see that the checks failed. Apparently, there's an issue with the commit name:
Could you fix it?
Thanks a lot!
|
Hello @afma-odoo everything should be good now, let me know! |
afma-odoo
force-pushed
the
19.0-masi-dian-enablement-doc-1
branch
from
abafce2 to
3c7c8b1
Compare
afma-odoo
left a comment
afma-odoo
left a comment
There was a problem hiding this comment.
Hi @masi-odoo, thank you for the updates.
I just force-pushed a minor typo that caused the documentation check to fail.
I approve it right away!
Can I r+ ?
Thanks
|
Hello @afma-odoo thanks for the typo help! It can be r+ then, thanks! |
|
@masi-odoo Great, thanks! @robodoo r+ |
|
@masi-odoo @afma-odoo because this PR has multiple commits, I need to know how to merge it:
|
|
Hello @afma-odoo seems there is an action needed by robodoo before merge. I did see something that changed, do you think it will be good to take out the "-" symbols or align them as bullet points? Maybe this would be the last improvement, but I could need your help on the one 😅 Best! 
|
afma-odoo
force-pushed
the
19.0-masi-dian-enablement-doc-1
branch
from
3c7c8b1 to
a5eaefe
Compare
|
Hello @masi-odoo! You're right, I fixed the "-" issue and force-pushed the changes Can you squash the commits? Thank you! |
…gestions DIAN enablement [ADD] accounting/l10n_co: adding automatic DIAN enablement process Apply suggestion from @masi-odoo [ADD] accounting/l10n_co: adding automatic DIAN enablement process - Added section on automatic DIAN enablement process for electronic invoicing, with new video reference - Eliminated previous video on the topic which is not up to date Co-Authored-By: afma-odoo <afma@odoo.com>
masi-odoo
force-pushed
the
19.0-masi-dian-enablement-doc-1
branch
from
a5eaefe to
334475c
Compare
|
All set @afma-odoo thanks! |
|
Thank you @masi-odoo @robodoo r+ |
3 participants
